Skip to content

Fix: Ally action link toggle [APP-2326] #497

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
DinetsV merged 1 commit into from
Jan 20, 2026
Merged

Fix: Ally action link toggle [APP-2326] #497

DinetsV merged 1 commit into from
Jan 20, 2026

Conversation

@bainternet
Copy link
Contributor

@bainternet bainternet commented Jan 20, 2026 •
edited by gitstream-cm bot
Loading

✨ PR Description

Purpose: Fix ally widget action to toggle open/close state instead of only opening, resolving APP-2326 issue with action link behavior.

Main changes:

  • Modified allyWidget:open action handler to check current widget state and toggle between open/close
  • Added conditional logic using isOpen() method to determine whether to open or close widget
  • Changed action behavior from always opening widget to toggling based on current state

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

@DinetsV DinetsV changed the base branch from master to release/v4.0.1 January 20, 2026 11:40
@DinetsV DinetsV merged commit c4e2f5a into release/v4.0.1 Jan 20, 2026
56 checks passed
@rami-elementor rami-elementor deleted the fix/APP-2326 branch January 20, 2026 12:27
bainternet added a commit that referenced this pull request Jan 20, 2026
@DinetsV @nirbhayel
@rami-elementor @bainternet
Release/v4.0.1 (#498)
2f647a2 
* [APP-0000] fix tab index

* fix: php error in legacy module [APP-2370] (#495)

* fix: php error in legacy module

* Fix fatal error

* Fix fatal error

---------

Co-authored-by: Rami Yushuvaev <[email protected]>

* Fix: Ally action link toggle [APP-2326] (#497)

* [APP-0000] add change log items

* readme.txt and README.MD

* Update readme.txt

Co-authored-by: Rami Yushuvaev <[email protected]>

---------

Co-authored-by: Nirbhay Singh <[email protected]>
Co-authored-by: Rami Yushuvaev <[email protected]>
Co-authored-by: Raz Ohad <[email protected]>
Co-authored-by: Rami Yushuvaev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rami-elementor rami-elementor rami-elementor approved these changes

@DinetsV DinetsV DinetsV approved these changes

Assignees

No one assigned

Labels

1 min review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bainternet @rami-elementor @DinetsV